Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DTRA] Maryia/DTRA-1559/feat: add isVerticalScrollEnabled prop to disable vertical scroll outside Y-axis #1625

Conversation

maryia-deriv
Copy link
Contributor

@maryia-deriv maryia-deriv commented Jul 31, 2024

To introduce the new isVerticalScrollEnabled prop which, when set to false, is working as follows:

  • on Wheel devices - disables pointer events on the chart when a user is scrolling vertically outside Y-axis (this approach is not applicable to touch devices),
  • on Touch devices - forces scroll on the nearest scrollable parent of the chart.

@maryia-deriv maryia-deriv changed the title [DTRA] Maryia/DTRA-1559/feat: disable onWheel event outside y-axis [DTRA] Maryia/DTRA-1559/feat: add isVerticalScrollEnabled prop to disable vertical scroll outside Y-axis Aug 1, 2024
@maryia-deriv maryia-deriv marked this pull request as ready for review August 1, 2024 15:05
(((date as unknown) as number) - bar.DT!.getTime()) /
(bar.DT!.getTime() - barPrev.DT!.getTime());
((date as unknown as number) - (bar.DT as Date).getTime()) /
((bar.DT as Date).getTime() - (barPrev.DT as Date).getTime());
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just fixing 'Forbidden non-null assertion' TS warning

Copy link

github-actions bot commented Aug 5, 2024

Preview Link: https://pr-1625-dtra.smartcharts-preview.pages.dev

Name Result
Build status Completed ✅
Preview URL Visit Preview
Action URL Visit Action

@balakrishna-deriv balakrishna-deriv merged commit ba6908f into deriv-com:master Aug 8, 2024
3 checks passed
@DerivFE
Copy link
Contributor

DerivFE commented Aug 8, 2024

🎉 This PR is included in version 2.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants